Skip to content

Fix the mc policy command following the 2019-07 release#47

Merged
sbesson merged 1 commit into
ome:masterfrom
sbesson:mc_policy_set
Aug 15, 2019
Merged

Fix the mc policy command following the 2019-07 release#47
sbesson merged 1 commit into
ome:masterfrom
sbesson:mc_policy_set

Conversation

@sbesson

@sbesson sbesson commented Aug 9, 2019

Copy link
Copy Markdown
Member

Travis has been failing since the breaking release of the mc utility (see https://travis-ci.org/ome/ome-common-java/jobs/565903545) -see https://github.com/minio/mc/releases/tag/RELEASE.2019-08-07T23-14-43Z and especially the breaking changes for the mc policy setting functionality

This change should update the start-location.sh code to use the newest mc policy set semantics and restore the build.

@sbesson sbesson requested a review from manics August 9, 2019 10:32
@sbesson sbesson changed the title Fix the mc policy command to match the 2091-08-07 changes Fix the mc policy command following the 2019-07 release of mc Aug 9, 2019
@sbesson sbesson changed the title Fix the mc policy command following the 2019-07 release of mc Fix the mc policy command following the 2019-07 release Aug 9, 2019
@joshmoore

Copy link
Copy Markdown
Member

👍 though this may point to pinning the version.

@sbesson

sbesson commented Aug 9, 2019

Copy link
Copy Markdown
Member Author

I wondered about the same. At the moment, we are downloading an alias to the latest version and we would need to use https://dl.minio.io/server/minio/release/linux-amd64/archive/ and https://dl.minio.io/client/mc/release/linux-amd64/archive/ for pinning minio/mc to a given version.

@sbesson

sbesson commented Aug 15, 2019

Copy link
Copy Markdown
Member Author

Merging to fix Travis CI on the master branch. If more breakages occur on the future, we can resurrect the pinning question.

@sbesson sbesson merged commit 9c239a3 into ome:master Aug 15, 2019
@sbesson sbesson deleted the mc_policy_set branch August 15, 2019 12:07
@sbesson sbesson mentioned this pull request May 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants